Skip to content

Updating UT workflow file to run on every push #968

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 6 commits into from
Feb 23, 2022
Merged

Updating UT workflow file to run on every push #968

merged 6 commits into from
Feb 23, 2022

Conversation

vrdmr
Copy link
Member

@vrdmr vrdmr commented Feb 23, 2022

Description

Fixes #

PR information

  • The title of the PR is clear and informative.
  • There are a small number of commits, each of which has an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
  • If applicable, the PR references the bug/issue that it fixes in the description.
  • New Unit tests were added for the changes made and CI is passing.

Quality of Code and Contribution Guidelines

@vrdmr vrdmr added the v3 label Feb 23, 2022
@codecov
Copy link

codecov bot commented Feb 23, 2022 •
edited
Loading

Codecov Report

Merging #968 (185f38d) into v3.x-dev (3755e68) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           v3.x-dev     #968   +/-   ##
=========================================
  Coverage     80.79%   80.79%           
=========================================
  Files            35       35           
  Lines          2302     2302           
  Branches        362      362           
=========================================
  Hits           1860     1860           
  Misses          335      335           
  Partials        107      107
Flag Coverage Δ
unittests 80.75% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3755e68...185f38d. Read the comment docs.

@vrdmr vrdmr force-pushed the vameru/v3-release-prep branch from f9c780c to ce8562d Compare February 23, 2022 07:20
@vrdmr vrdmr force-pushed the vameru/v3-release-prep branch from ce8562d to 185f38d Compare February 23, 2022 07:34
@vrdmr vrdmr merged commit a7a928b into v3.x-dev Feb 23, 2022
@vrdmr vrdmr deleted the vameru/v3-release-prep branch February 23, 2022 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gavin-aguiar gavin-aguiar gavin-aguiar approved these changes

@YunchuWang YunchuWang Awaiting requested review from YunchuWang YunchuWang is a code owner

Assignees
No one assigned
Labels
v3
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vrdmr @gavin-aguiar